Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

platform: how to establish peering connection between Aiven and UpCloud #2163

Merged
merged 15 commits into from
Oct 4, 2023

Conversation

wojcik-dorota
Copy link
Contributor

@wojcik-dorota wojcik-dorota self-assigned this Sep 28, 2023
@wojcik-dorota wojcik-dorota changed the title platform: how to establish peering connection between Aiven and UpCloud [WiP] platform: how to establish peering connection between Aiven and UpCloud Sep 28, 2023
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 28, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: a066f38
Status: ✅  Deploy successful!
Preview URL: https://924db3b4.devportal.pages.dev
Branch Preview URL: https://dorota-platform-upcloud-peer.devportal.pages.dev

View logs

@wojcik-dorota wojcik-dorota changed the title [WiP] platform: how to establish peering connection between Aiven and UpCloud platform: how to establish peering connection between Aiven and UpCloud Sep 29, 2023
@wojcik-dorota wojcik-dorota marked this pull request as ready for review September 29, 2023 15:03
Copy link
Contributor

@giacomo-alzetta-aiven giacomo-alzetta-aiven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The view on the peering list was updated recently so point 6 in "Aiven VPC network UUID" needs to be updated. Some other minor notes.

In general the instructions look correct to me 👍

docs/platform/howto/vpc-peering-upcloud.rst Show resolved Hide resolved
docs/platform/howto/vpc-peering-upcloud.rst Outdated Show resolved Hide resolved
docs/platform/howto/vpc-peering-upcloud.rst Outdated Show resolved Hide resolved
docs/platform/howto/vpc-peering-upcloud.rst Outdated Show resolved Hide resolved
docs/platform/howto/vpc-peering-upcloud.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@giacomo-alzetta-aiven giacomo-alzetta-aiven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! I will give @sjamgade some time to look at this too before merging.

@sjamgade sjamgade merged commit f5fc07e into main Oct 4, 2023
5 checks passed
@sjamgade sjamgade deleted the dorota-platform-upcloud-peering-setup branch October 4, 2023 09:31
@sjamgade
Copy link
Contributor

sjamgade commented Oct 4, 2023

this should probably have been a squash and merge,

@wojcik-dorota
Copy link
Contributor Author

this should probably have been a squash and merge,

@sjamgade we don't have a strict squashing policy in devportal so all good :-) Thank you!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants